Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Youtube: add plugin for viewing videos/channels #6740

Merged
merged 11 commits into from
May 26, 2020
Merged

Conversation

mia-pi-git
Copy link
Member

@mia-pi-git mia-pi-git commented May 24, 2020

This is mostly for the youtube room. However, /requestapproval and /link are meant for all rooms, since they're very commonly used through Kid A and very useful, and can be streamlined a bit here (they're in this plugin so they can make use of the Youtube interface.)
Specifics are here https://www.smogon.com/forums/threads/youtube-channel-database.3664726/.
This has been approved by both Kie and A Random Duck.
As for the Config setting for the api key, i have a key that can be used, and can share privately for someone who can set that.

Copy link
Member

@KrisXV KrisXV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I personally think the requestapproval/link function that is ported from Kid A should be in a separate commit.

server/chat-plugins/youtube.ts Outdated Show resolved Hide resolved
server/chat-plugins/youtube.ts Outdated Show resolved Hide resolved
server/chat-plugins/youtube.ts Outdated Show resolved Hide resolved
server/chat-plugins/youtube.ts Outdated Show resolved Hide resolved
mia-pi-git and others added 2 commits May 24, 2020 15:22
Co-authored-by: Kris Johnson <11083252+KrisXV@users.noreply.github.com>
@mia-pi-git
Copy link
Member Author

I personally think the requestapproval/link function that is ported from Kid A should be in a separate commit.

I felt it fit here, since it uses YoutubeInterface functions, but if it's desired, I can move it out of this PR.

@mia-pi-git mia-pi-git requested a review from Asheviere May 25, 2020 05:27
@mia-pi-git
Copy link
Member Author

mia-pi-git commented May 25, 2020

(requesting since they ran the plugin on the bot, so i thought input from them might be helpful - lmk if I shouldn't have done that)

@mia-pi-git
Copy link
Member Author

Removed approvals/link for now.

@Zarel
Copy link
Member

Zarel commented May 26, 2020

Looks good to me!

@Zarel Zarel merged commit de24f97 into smogon:master May 26, 2020
@mia-pi-git mia-pi-git deleted the youtube branch May 26, 2020 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants